Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Erandtke/210928 map salting #22

Open
wants to merge 13 commits into
base: knight
Choose a base branch
from
Open

Conversation

erandtkecr
Copy link

Want to contribute? Great! Make sure you've read, understood and considered all
the points below before creating your PR:

  • Keep your PR under 200 lines of code and address a single concern.
  • Add unit test(s) and documentation (these do not count toward your 200 lines).
  • Adhere to the Google C++ Style Guide.
  • Run ninja test or catkin_make_isolated --install --use-ninja --pkg cartographer --make-args test as appropriate.
  • Keep rebasing (or merging) of master branch to a minimum. It triggers Travis
    runs for every update which blocks merging of other changes. Our merge bot
    will rebase your branch, reformat your source code and merge as the last step
    in the review process.
  • Please replace this template text with the commit message you want for your
    PR. You and/or the reviewer should keep it updated during the course of the
    review using the GitHub edit feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant